Skip to content
This repository has been archived by the owner on May 24, 2021. It is now read-only.

[Work in progress]-DoliWoo debug and improvement #88

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

philippe-opendsi
Copy link

@philippe-opendsi philippe-opendsi commented Aug 14, 2016

Hi,
This PR contains several update to Doliwoo.

  • Enable or disable syncing of photos
  • Ability to set desired price level to use in the store --> need an update to Dolibarr's webservices (file server_productorservice.php ). I will make a pull request quickly. You can download his file here
  • Enable or disable order's sync
  • Try to add some enhancement on customer creation

  •  General Comment
  • <img border=0 src='https://avatars.githubusercontent.com/u/1210367?v=3' height=16 width=16'> Thanks Philippe.
    I'd like to review your changes but I won't bother until the integration tests pass.
    Can you please fix the errors?
    The log is at https://travis-ci.org/GPCsolutions/doliwoo/builds/152219078
    You can run these tests locally by installing grunt and launching grunt test from the project root. (I may have to add a note about that in the readme…).
    Cheers,

  

cmoinon and others added 7 commits July 7, 2016 10:44
…ce Trying to get property of non-object in /var/www/wpwoocommerce/wp-content/plugins/doliwoo/doliwoo.php on line 214
…ce or multiprice ou multiprice_ttc depending of a parameter in doliwoo dolibar_price_level add
@rdoursenaud
Copy link
Member

Thanks Philippe.

I'd like to review your changes but I won't bother until the integration tests pass.

Can you please fix the errors?
The log is at https://travis-ci.org/GPCsolutions/doliwoo/builds/152219078

You can run these tests locally by installing grunt and launching grunt test from the project root. (I may have to add a note about that in the readme…).

Cheers,

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants